Greptile Summary

This PR threads a log.Logger through the HeightSync factory and both TxStore implementations (LegacyPool and CosmosTxStore), adds a log line in TxStore.Txs() reporting how many transactions were collected vs. stored, and introduces two new observability additions: a height_sync/heights meter metric in HeightSync.EndCurrentHeight() and a total counter field on TxStore to back the log.

Key changes:

• HeightSync.reset signature updated to func(log.Logger) *Store; a logger is stored on the struct and forwarded to every new store created at height transitions.
• legacypool.New gains a logger parameter; a tagged sub-logger (pool=legacypool) is passed to HeightSync.
• NewTxStore (EVM) accepts and uses a logger, tagging it with txstore=evm.
• NewCosmosTxStore accepts (and intentionally ignores) a logger to satisfy the updated interface.
• TxStore gains a total uint64 field incremented in AddTxs and decremented in RemoveTx; the value is surfaced in the new Info-level log.
• Two issues found: (1) the Txs() log uses Info level in a per-block hot path, risking log flooding in production; (2) AddTxs checks t.lookup for deduplication but never populates it, meaning total will overcount when the same transaction is re-submitted.

Confidence Score: 3/5

• Safe to merge with low operational risk, but the total_in_store metric logged on every block will be inaccurate due to a missing lookup population in AddTxs, and the Info-level log may flood production logs.
• The core logic changes (metric registration, logger threading, signature updates) are all correct and well-scoped. However, the new total counter is built on top of a pre-existing deduplication gap (lookup never written in AddTxs), making the logged value unreliable. Additionally, logging at Info in Txs() — a per-block hot path — is a meaningful operational concern for production deployments.
• mempool/txpool/legacypool/tx_store.go — the total counter accuracy and log level both need attention before this lands in production.

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant BP as Block Proposer
    participant HS as HeightSync[TxStore]
    participant TS as TxStore
    participant M as Metrics

    Note over HS: StartNewHeight(h)
    HS->>TS: reset(logger) → new TxStore
    HS->>M: (height change signalled)

    Note over HS: EndCurrentHeight()
    HS->>M: hsHeights.Mark(1)
    HS->>HS: close(done)

    BP->>HS: GetStore(ctx, h)
    HS-->>BP: *TxStore (after done / ctx timeout)

    BP->>TS: Txs(filter)
    TS->>TS: filter & convert txs
    TS->>TS: logger.Info("collected txs", total_in_store, num_selected)
    TS->>M: txsCollected.Mark(numSelected)
    TS-->>BP: map[addr][]*LazyTransaction

_{Last reviewed commit: 7a5b275}

Codecov Report

❌ Patch coverage is 50.00000% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 63.91%. Comparing base (bea1ab9) to head (3b49984).
⚠️ Report is 1 commits behind head on main.

@@            Coverage Diff             @@
##             main    #1071      +/-   ##
==========================================
- Coverage   64.11%   63.91%   -0.20%     
==========================================
  Files         331      331              
  Lines       23303    23316      +13     
==========================================
- Hits        14941    14903      -38     
+ Misses       6946     6944       -2     
- Partials     1416     1469      +53